Cleanup of SimpleSeleniumTestCase. Removed unnecessary include.
authorPriyanka Dhanda <pdhanda@users.mediawiki.org>
Thu, 14 Oct 2010 17:56:52 +0000 (17:56 +0000)
committerPriyanka Dhanda <pdhanda@users.mediawiki.org>
Thu, 14 Oct 2010 17:56:52 +0000 (17:56 +0000)
maintenance/tests/selenium/SimpleSeleniumConfig.php
maintenance/tests/selenium/SimpleSeleniumTestCase.php
maintenance/tests/selenium/SimpleSeleniumTestSuite.php

index 48de1e2..cffa83c 100644 (file)
@@ -3,7 +3,7 @@ class SimpleSeleniumConfig {
        
        public static function getSettings(&$includeFiles, &$globalConfigs) {
                $includes = array(
-                       'skins/Chick.php'
+                       //files that needed to be included would go here
                );
                $configs = array(
                        'wgDefaultSkin' => 'chick'
index 8f27dcd..8f01b43 100644 (file)
@@ -17,6 +17,10 @@ class SimpleSeleniumTestCase extends SeleniumTestCase {
                $this->assertEquals( $correct, true );
        }
        
+       /*
+        * All this test really does is verify that a global var was set.
+        * It depends on $wgDefaultSkin = 'chick'; being set
+        */
        public function testGlobalVariableForDefaultSkin() {
                $this->open( $this->getUrl() . '/index.php?&action=purge' );
                $bodyClass = $this->getAttribute( "//body/@class" );
index cf32937..49a06d4 100644 (file)
@@ -7,7 +7,6 @@
  * $wgSeleniumTestConfigs['SimpleSeleniumTestSuite'] = 'SimpleSeleniumConfig::getSettings';
  * OR
  * 2) Add the following to your Localsettings.php
- * require_once( "$IP/skins/Chick.php" );
  * $wgDefaultSkin = 'chick';
  */
 class SimpleSeleniumTestSuite extends SeleniumTestSuite